Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

postgres95: Bugfix, Make PostgreSQL work with newer libxml2 #26726

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

postgres95: Bugfix, Make PostgreSQL work with newer libxml2 #26726

wants to merge 1 commit into from

Conversation

essandess
Copy link
Contributor

Fixes:

Description

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS x.y
Xcode x.y / Command Line Tools x.y.z

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@macportsbot
Copy link

Notifying maintainers:
@jyrkiwahlstedt for port postgresql95.

@barracuda156
Copy link
Contributor

@essandess Can’t this be added to all other 9x as well? They will need openssl.branch 1.0 in addition, I think.

@essandess
Copy link
Contributor Author

@essandess Can’t this be added to all other 9x as well? They will need openssl.branch 1.0 in addition, I think.

There are other issues that break postgresql94 and previous. The configure phase doesn't even work.

@barracuda156
Copy link
Contributor

@essandess Can’t this be added to all other 9x as well? They will need openssl.branch 1.0 in addition, I think.

There are other issues that break postgresql94 and previous. The configure phase doesn't even work.

Configure does not work, because no one bothered to set the required OpenSSL there.

@essandess
Copy link
Contributor Author

@essandess Can’t this be added to all other 9x as well? They will need openssl.branch 1.0 in addition, I think.

There are other issues that break postgresql94 and previous. The configure phase doesn't even work.

Configure does not work, because no one bothered to set the required OpenSSL there.

I tried adding openssl.branch 1.0, but the configure issue is unrelated: something about not being able to check that long int or long long int's are 64 bit (!).

@barracuda156
Copy link
Contributor

@essandess Hmm, okay. I have built postgresql92 just a few days ago after its revbump, and all I needed was openssl fix, but that was on 10.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jyrkiwahlstedt jyrkiwahlstedt

Labels
maintainer: requires approval maintainer: timeout
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@essandess @macportsbot @barracuda156 @jyrkiwahlstedt